Skip to content

Conversation

@oscargus
Copy link
Contributor

@oscargus oscargus commented Jul 3, 2025

We've had requests to add the Surfer waveform viewer to OSS CAD Suite, so here goes. No idea if it works, but at least looks promising locally.

Based on discussions with @mattvenn I can inform that the pre-built Linux binary that we offer is 15.5 MB and surver 1 MB (the remote server version). I think this should provide both.

Surfer requires Rust 1.82 and there may be openSSL issues as with #155, but we do not explicitly require any give version, but maybe the crate that we are using does.

@mmicko
Copy link
Member

mmicko commented Jul 3, 2025

Thanks @oscargus ,
Have need to do some changes in build script (surfer.sh) to just make it at least build but would suggest you try using
./builder.py build --target=surfer --arch=linux-x64
So to at get at least of linux-x64 compiling and installing binary in output directory.
For macOS I will need to find a proper solution, but would be of help it you can at least do this part.

@oscargus
Copy link
Contributor Author

oscargus commented Jul 4, 2025

Sure, just have to set up docker properly (I go not get it to work on WSL which is what I have access to at the moment). Can probably sort of macos as well.

@oscargus oscargus marked this pull request as draft July 10, 2025 14:35
@oscargus
Copy link
Contributor Author

Converting this to draft since I need my office machines and I'll try to have some vacation. Will get back in a month or so.

@oscargus oscargus marked this pull request as ready for review July 11, 2025 14:44
@oscargus
Copy link
Contributor Author

Turns out that I had to go to the office anyway. Now it should be working, I think for all platforms. My bad for not understanding that cargo install didn't work out of the box.

@mmicko
Copy link
Member

mmicko commented Jul 11, 2025

Thanks. Will update few things after merge and will add CI tasks

@mmicko mmicko merged commit 27c59a2 into YosysHQ:main Jul 11, 2025
@mmicko
Copy link
Member

mmicko commented Jul 11, 2025

@oscargus Thanks for working on this. linux-arm64 is not working due to openssl, so have to see what exactly is causing this, disabled for now. Also disabled LTO on main executable, I know it will give not optimal results but it takes too much memory that could be a problem on CI.
Have started CI so we will see how it ends :)

@mmicko
Copy link
Member

mmicko commented Jul 11, 2025

@oscargus There are some issues starting surfer if I am using wrapped binaries (that lets it run on any linux), so for now I will let it use system libraries since only thing used is dynamically loaded graphics drivers, but anyway would like to figure it out if this works fine for everyone or we need some changes. Please note that I will not be available next two weeks but now you are aware how to check compilation at least. I can merge things if you have some fixes and did try it out, but no way of checking anything. Soon all the builds will be done, so please do try if it is working on your machines.

Thanks,
Micko

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants